[PATCH] gas: Revert PR 32391 related commits to fix 3 regressions

From hjl.tools@gmail.com  Thu Dec 26 08:12:36 2024
Message-ID: <20241226081236.495980-1-hjl.tools@gmail.com>

9f2e3c21f65 Fix the handling or arguments and macro pseudo-variables inside nested assembler macros.

introduced 3 regressions of PR gas/32484, PR gas/32486 and PR gas/32487.
Revert all PR 32391 related commits and add tests for PR gas/32484,
PR gas/32486, PR gas/32487.

PR gas/32484
PR gas/32486
PR gas/32487
* testsuite/gas/macros/macros.exp: Run nesting1, nesting2 and
nesting3.

Signed-off-by: H.J. Lu <hjl.tools@gmail.com>
Gbp-Pq: Name pr32391-revert.diff

[PATCH RESEND] gold: Support percent-encoded JSON in --package-metadata

From benjamin.drung@canonical.com  Tue Dec 17 15:27:46 2024
Message-ID: <20241217152746.98360-1-benjamin.drung@canonical.com>

Specifying the compiler flag `-Wl,--package-metadata=<JSON>` will not
work in case the JSON contains a comma, because compiler drivers eat
commas. Example:

```
$ echo "void main() { }" > test.c
$ gcc -fuse-ld=gold '-Wl,--package-metadata={"type":"deb","os":"ubuntu"}' test.c
/usr/bin/ld.gold: error: cannot open "os":"ubuntu"}: No such file or directory
/usr/bin/ld.gold: fatal error: error: --package-metadata={"type":"deb" does not contain valid JSON: '}' expected near end of file

collect2: error: ld returned 1 exit status
```

The quotation marks in the JSON value do not work well with shell nor
make. Specifying the `--package-metadata` linker flag in a `LDFLAGS`
environment variable might loose its quotation marks when it hits the
final compiler call.

Following the same format as the implementation in ld:
b0cc81e87087bb8a6b12dc1e4fd7f2591927977b

So support percent-encoded and %[string] encoded JSON data in the
`--package-metadata` linker flag. Percent-encoding is used because it is
a standard, simple to implement, and does take too many additional
characters. %[string] encoding is supported for having a more readable
encoding.

Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=32003
Bug-Ubutru: https://bugs.launchpad.net/bugs/2071468
Signed-off-by: Benjamin Drung <benjamin.drung@canonical.com>
Gbp-Pq: Name gold-package-metadata.diff

armhf-64k-alignment

The upstream binutils commit below reduced the section alignment of
armhf executables (including shared libraries) from 64k to 4k, on the
basis that the additional bloat is not justified, given that arm64 hosts
running 64k pagesize kernels with armhf user space are not used in
practice.

However, the situation is slightly different today on Debian, where a
16k pagesize kernel is provided that enables compat support, and is
therefore able to execute armhf user space. The only impediment is the
fact that armhf binaries are no longer compatible with this, due to the
change below.

To accommodate this use case, revert the patch. This makes the generated
executables compatible with compat mode on arm64 systems running 16k
kernels.

Gbp-Pq: Name armhf-64k-alignment.diff

pr32392-workaround

Gbp-Pq: Name pr32392-workaround.diff

env-package-metadata

Gbp-Pq: Name env-package-metadata.diff

perl-shebang

Gbp-Pq: Name perl-shebang.diff

mips-hack

Gbp-Pq: Name mips-hack.diff

libctf-soname

Gbp-Pq: Name libctf-soname.diff

gold-no-keep-files-mapped

# DP: gold on 32bit archs: Release mapped files after each pass.

# DP: gold on 32bit archs: Release mapped files after each pass.

Gbp-Pq: Name gold-no-keep-files-mapped.diff

aarch64-libpath

Gbp-Pq: Name aarch64-libpath.diff

gprof-build

# DP: Fix gprof build error.

# DP: Fix gprof build error.

Gbp-Pq: Name gprof-build.diff

_ld_doc_remove_xref

# DP: In ld.texi, remove cross reference to BFD internals documentation.

# DP: In ld.texi, remove cross reference to BFD internals documentation.

Gbp-Pq: Name 164_ld_doc_remove_xref.diff

_gold_dummy_zoption

# DP: let gold accept a dummy -z buildd-<random-string> option.

# DP: let gold accept a dummy -z buildd-<random-string> option.

Gbp-Pq: Name 161_gold_dummy_zoption.diff

_ar_scripts_with_tilde

Gbp-Pq: Name 157_ar_scripts_with_tilde.patch

Build libbfd with -fPIC to allow linking with PIE binaries

Gbp-Pq: Name 136_bfd_pic.patch

_bfd_soversion

Gbp-Pq: Name 135_bfd_soversion.patch

Description: Fix ld-bootstrap testsuite when configured with --enable-plugins

Upstream status: proposed patch

Upstream status: proposed patch

Gbp-Pq: Name 131_ld_bootstrap_testsuite.patch

Description: Disable build of gold/testsuite

Upstream status: local

Upstream status: local

Gbp-Pq: Name 130_gold_disable_testsuite_build.patch

_multiarch_libpath

# DP: Add multiarch directories to linker search path for ld and gold.

# DP: Add multiarch directories to linker search path for ld and gold.

Gbp-Pq: Name 129_multiarch_libpath.patch

_ppc64_powerpc_biarch

Gbp-Pq: Name 128_ppc64_powerpc_biarch.patch

Explicitly use bash for the ld testsuite.

Gbp-Pq: Name 013_bash_in_ld_testsuite.patch

Specify which filename is causing an error if the filename is a

directory. (#45832)

Gbp-Pq: Name 006_better_file_error.patch

Don't mention monitor(3) which doesn't exist in Debian. (#160654)

Gbp-Pq: Name 003_gprof_see_also_monitor.patch

Add more documentation about profiling and -fprofile-arcs.

Gbp-Pq: Name 002_gprof_profile_arcs.patch

Description: correct where ld scripts are installed

Upstream status: N/A

Upstream status: N/A
Date: ??

Gbp-Pq: Name 001_ld_makefile_patch.patch

binutils (2.43.50.20241230-1) unstable; urgency=medium

  * New upstream snapshot, taken from the trunk.
  * Revert the fix for PR gas/32391. Closes: #1090999, #1091488.
  * Build with GCC 14.2.0-12 on ARM32. Closes: #1091281.

[dgit import unpatched binutils 2.43.50.20241230-1]

Import binutils_2.43.50.20241230.orig.tar.xz

[dgit import orig binutils_2.43.50.20241230.orig.tar.xz]

Import binutils_2.43.50.20241230-1.debian.tar.xz

[dgit import tarball binutils 2.43.50.20241230-1 binutils_2.43.50.20241230-1.debian.tar.xz]